Skip to content

7.x: Fix console on bare-metal installs#2037

Merged
baszoetekouw merged 1 commit into
OpenConext:mainfrom
tvdijen:patch-1
Jul 6, 2026
Merged

7.x: Fix console on bare-metal installs#2037
baszoetekouw merged 1 commit into
OpenConext:mainfrom
tvdijen:patch-1

Conversation

@tvdijen

@tvdijen tvdijen commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Open for discussion, because I don't know if this is the right solution, but we ran into the issue that the .env file was not loaded automatically when running bin/console commands. We run multiple instances of EB on the same machine, so it's crucial that we can set the environment variables (especially APP_SECRET) per instance instead of setting it globally. This change seems to fix that.

Open for discussion, because I don't know if this is the right solution, but we ran into the issue that the .env file was not loaded automatically when running `bin/console` commands.
We run multiple instances of EB on the same machine, so it's crucial that we can set the environment variables (especially APP_SECRET) per instance instead of setting it globally.  This change seems to fix that.
@tvdijen tvdijen changed the title Fix console on bare-metal installs 7.x: Fix console on bare-metal installs Jun 30, 2026
@baszoetekouw baszoetekouw requested a review from johanib July 2, 2026 07:11

@johanib johanib left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I took a look, this seems a fine fix 👍 It's narrowly scoped, so should not break anything.

As a sidenote, I noticed a bug in the dotenv not loading .env override files, for example .env.prod is ignored. Made a separate pr for that: #2042

@baszoetekouw baszoetekouw changed the base branch from release/7.1 to main July 6, 2026 08:53
@baszoetekouw baszoetekouw merged commit cc6fd21 into OpenConext:main Jul 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants